Skip to content

ci/kas-container-shell-helper: run with the build's kas configuration - #3109

Merged
Anuj Mittal (anujm1) merged 1 commit into
qualcomm-linux:masterfrom
ricardosalveti:ci-bitbake-startup
Sep 11, 2026
Merged

ci/kas-container-shell-helper: run with the build's kas configuration#3109
Anuj Mittal (anujm1) merged 1 commit into
qualcomm-linux:masterfrom
ricardosalveti:ci-bitbake-startup

Conversation

@ricardosalveti

Copy link
Copy Markdown
Contributor

The helper always starts kas with ci/base.yml, so the buildstats step between a job's world and image builds rewrites bblayers.conf for a smaller layer set. bitbake then treats its parse cache as stale and the image build parses all 3851 recipes again instead of loading them, 40 s against about 10 s for a cache hit, on each of the 36 nightly jobs that build world.

Use KAS_YAMLS when the caller exports it, as compile.yml does, and keep ci/base.yml as the default for the layer checks.

The helper always starts kas with ci/base.yml, so the buildstats step
between a job's world and image builds rewrites bblayers.conf for a
smaller layer set. bitbake then treats its parse cache as stale and the
image build parses all 3851 recipes again instead of loading them, 40 s
against about 10 s for a cache hit, on each of the 36 nightly jobs that
build world.

Use KAS_YAMLS when the caller exports it, as compile.yml does, and keep
ci/base.yml as the default for the layer checks.

Assisted-by: Claude Code:claude-fable-5
Signed-off-by: Ricardo Salveti <ricardo.salveti@oss.qualcomm.com>
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 1a392b2

qcom-distro
Pass: 312 | Fail: 1 | Total: 357
qcom-distro_linux-qcom-6.18
Pass: 237 | Fail: 0 | Total: 259
nodistro
Pass: 10 | Fail: 0 | Total: 10

@test-reporting-app

test-reporting-app Bot commented Sep 10, 2026

Copy link
Copy Markdown

Test Results

  119 files  +   51    715 suites  +396   6h 20m 58s ⏱️ - 5h 16m 5s
  171 tests +   18    169 ✅ +   56   1 💤  -  2  1 ❌  - 36 
4 605 runs  +2 526  4 538 ✅ +2 544  66 💤 +48  1 ❌  - 66 

For more details on these failures, see this check.

Results for commit 1a392b2. ± Comparison against base commit bc1ff3a.

This pull request removes 4 and adds 22 tests. Note that renamed tests count towards both.
lava ‑ auto-login-action
lava ‑ lava-test-retry
lava ‑ lava-test-shell
lava ‑ minimal-boot
10_Logging_Journalctl_Validation ‑ Logging_Journalctl_Validation
11_Partition_PostBoot_Validation ‑ Partition_PostBoot_Validation
12_Boot_Systemd_KPI_Loop ‑ Boot_Systemd_KPI_Loop
5_OpenCV ‑ OpenCV
6_Ethernet ‑ Ethernet
6_irq ‑ irq
7_BT_ON_OFF ‑ BT_ON_OFF
7_Docker_Kernel_Config ‑ Docker_Kernel_Config
8_AudioRecord ‑ AudioRecord
8_EFI_Variable_Validation ‑ EFI_Variable_Validation
…

♻️ This comment has been updated with latest results.

@koenkooi Koen Kooi (koenkooi) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit message needed a 3rd read before I caught on, but the change is good.

@qcomlnxci

Copy link
Copy Markdown

Test Coral run workflow

Test jobs for commit 1a392b2

  • qcomdistro: multimedia image-prop
    Pass: 40 | Fail: 0 | Total: 40
  • qcomdistro: multimedia image
    Pass: 9 | Fail: 0 | Total: 9

@anujm1
Anuj Mittal (anujm1) merged commit 2e2c53e into qualcomm-linux:master Sep 11, 2026
242 of 243 checks passed
@quic-yocto-ci

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants